Skip to content

ci: skip scheduled and release workflows on forks #8719

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 9 commits into from
Nov 20, 2025
Merged

ci: skip scheduled and release workflows on forks #8719

merged 9 commits into from
Nov 20, 2025
+18 −0

Conversation

@joffeoja
Copy link
Contributor

@joffeoja joffeoja commented Nov 13, 2025 •
edited
Loading

Description

Prevent GitHub Actions workflows that rely on repository secrets or publishing permissions from running on forks. A lot of daily/weekly email spam from failed workflows on forks that are missing the secrets...

This change updates multiple workflows so they only run when the repository is continuedev/continue by adding a guard just like in https://github.com/continuedev/continue/blob/main/.github/workflows/cla.yaml#L17-L18:

if: github.repository == 'continuedev/continue'

This ensures that scheduled tasks, release automation, metrics collection, and dependency submission workflows do not run (and fail) on contributor forks, while preserving all existing behavior for the main repository.

Workflows updated include:

  • Daily Snyk Agent
  • Metrics (issue metrics)
  • GitHub dependency graph submission
  • Beta Release
  • Stable Release
  • Package release workflows for:
    • @continuedev/config-yaml
    • @continuedev/fetch
    • @continuedev/llm-info
    • @continuedev/openai-adapters

Fixes #8662.

Tests

  • Validated syntax for each updated workflow
  • Confirmed that workflows are skipped on forks
  • Verified that workflows continue to run normally on continuedev/continue

@joffeoja joffeoja changed the title Skip run on forks ci: skip scheduled and release workflows on forks Nov 13, 2025
@joffeoja joffeoja marked this pull request as ready for review November 13, 2025 21:29
@joffeoja joffeoja requested a review from a team as a code owner November 13, 2025 21:29
@joffeoja joffeoja requested review from sestinj and removed request for a team November 13, 2025 21:29
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Nov 13, 2025
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Nov 13, 2025
View reviewed changes
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 9 files

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Nov 19, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 19, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Nov 19, 2025

@joffeoja looks like you may need to rebase with main, a bunch of test failures have been resolved on main.

@joffeoja
Copy link
Contributor Author

joffeoja commented Nov 19, 2025

Hi @RomneyDa, I just rebased and force pushed my branch to avoid any extra merges. Looks like you have made good progress on the flaky tests, but still some are failing. None of those are related to my changes though, I'm hoping for a calmer inbox when they get merged or I have to delete the fork until next time :)

@RomneyDa
Copy link
Collaborator

RomneyDa commented Nov 20, 2025

@joffeoja looks great, thank you for the updates. I'll double check with @sestinj. The remaining failures are non-blocking CLI flake

@RomneyDa RomneyDa merged commit 978a79d into continuedev:main Nov 20, 2025
53 of 56 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 20, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 20, 2025
@joffeoja joffeoja deleted the skip_run_on_forks branch November 20, 2025 21:37
@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.31.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.35.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sestinj
Copy link
Contributor

sestinj commented Nov 20, 2025

🎉 This PR is included in version 1.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

@RomneyDa RomneyDa RomneyDa approved these changes

@sestinj sestinj Awaiting requested review from sestinj sestinj is a code owner automatically assigned from continuedev/continue-code-reviewers

Assignees

No one assigned

Labels

lgtm This PR has been approved by a maintainer released size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Issues and PRs
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Disable beta-release + snyk workflows on Github forks

3 participants

@joffeoja @RomneyDa @sestinj